Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A few fixes to support Django 3+ and latest version of TelegramBot #39

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

leikoilja
Copy link

A few changes to update the example code as well as making the TelegramError display a better readable errors as well as adding use_context to bypass deprecation messages.
Also adding an option to set the handler's file's custom name.

@jafrancov
Copy link

Just wondering if this pull requests will be merged, with the updates of Python Telegram Bot, this will solve all the warnings and get it back working again.

@leikoilja
Copy link
Author

leikoilja commented Nov 3, 2021

@JungDev can you please take a look at these? :)

@jafrancov, to be honest, i am afraid @JungDev does not seem to be active on github anymore. We are probably better of creating our own fork and maintaining it :)

@jafrancov
Copy link

@leikoilja That's what I thought, and I did the fork, we are working in a new implementation, but we have to make the changes in the library itself to make it work, I was waiting for the merge, but, I did notice the absence of @JungDev, so... fork will be the answer, thanks!

@Ali-Aref
Copy link

is there any fork of this which is on maintenance ? let's create other main one (that's better Instate of everyone with his own fork)

@lukruh
Copy link

lukruh commented Dec 20, 2021

I also created a fork, which I use for my bots. I'm not actively maintaining it and have no time for frequent updates, but I'm active on Github and will have a look at PR. I think a shared fork would be nice. I'd prefer to create some PR from time to time rather than maintaining the repo ;).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants